-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature (296629): Faire la page de notifications #136
feature (296629): Faire la page de notifications #136
Conversation
Signed-off-by: Jean-Christophe Drouin <[email protected]>
Signed-off-by: Jean-Christophe Drouin <[email protected]>
|
||
<TouchableOpacity | ||
style={[styles.tab, activeTab === 'Historique' && styles.activeTab]} | ||
onPress={() => setActiveTab('Historique')} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Utiliser setActiveTab(t('Screens.History') genre comme en haut
fontSize: TextTheme.labelTitle.fontSize, | ||
fontWeight: TextTheme.labelTitle.fontWeight, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu peux le remplace par ...Texttheme.labelTitle
{t('Screens.Notifications')} | ||
</Text> | ||
{notificationCount > 0 && ( | ||
<View style={styles.badge}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Est-ce qu'il ne faudrait pas le positionner en position absolue? De quoi ça à l'air sur une tablet le badge?
return sections | ||
} | ||
|
||
type sectionType = { title: string; data: NotificationReturnType } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SectionType
const styles = StyleSheet.create({ | ||
separator: { | ||
borderBottomWidth: 1, | ||
borderBottomColor: ColorPallet.grayscale.lightGrey, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#DAE6F0
fontSize: TextTheme.labelTitle.fontSize, | ||
fontWeight: TextTheme.labelTitle.fontWeight, | ||
color: TextTheme.labelTitle.color, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu peux le remplacer par ...TextTheme.labelTitle
}, | ||
sectionSeparator: { | ||
height: 1, | ||
backgroundColor: ColorPallet.grayscale.lightGrey, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#DAE6F0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarde mes commentaires
Signed-off-by: Jean-Christophe Drouin <[email protected]>
No description provided.